I agree with the proposal but with the following remarks:

• We should also add clock_safe and perhaps rename it.
• Fallible and active is statically true or false only
• self_sync should be either true/false statically or true when active (active here meaning when the source is being used)

I understand that some of these sources have parameters to change these values but I believe they should be "default" values, indicating the default behavior the user can expect.

We should accompany this PR with a new page in the online doc that explains what these entries mean too. I'm down to write it.

Is it better now?

Is it better now?

Yes!

I'm working on this. I should have mentioned that those flags really mean something on sources. Not sure how to convey how operators work..

@toots I filled in the parameters I could, but I would need a serious review on this one: I am far from being sure that I filled everything correctly or that I did not miss something...

A way to review is

LIQ_FREI0R=false LIQ_LADSPA=false ./liquidsoap --no-stdlib --list-functions-md 

At the end of the doc of operators there is now something like

This function is passive, fallible, not self-syncing and clock-safe.

Will have a look shortly, thanks!

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sorry for the lack of response here. Still down to consider it once rebased.